Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Frida #487

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Frida #487

wants to merge 2 commits into from

Conversation

fridaforser
Copy link

Copy link

@AntonellaMorittu AntonellaMorittu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Frida, congrats deploying your first DB! The code looks well structured an I see no issues in it, but I couldn't test all the endpoints, I gave an internal error back after trying several times. I can't see from the code how was the DB populated, did you test it and did you manage to get the right responses back? Regardless, your implementation meets the basic requirements so you passed! 🥂

@AntonellaMorittu
Copy link

btw I checked it again and it works very well, probably Render was being difficult! :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants